Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modifications to get it to work on Ubuntu Server 20.04 on 64 bit ARM #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

WillooWisp
Copy link

Modifications to get it to work on 64 bit CPU, where PROT_EXEC had to be removed. There is a PR for this at joan2937/pigpio#342.

Removed -a 1 flags from pigpiod since it gives errors on Ubuntu Server 20.04 on 64 bit ARM.

…be removed. There is a PR for this at joan2937/pigpio#342.

Removed -a 1 flags from pigpiod since it gives errors on Ubuntu Server 20.04 on 64 bit ARM.
@corbosman
Copy link
Owner

Does this not mean it'll stop working on other platforms? I had to specifically use -a 1 on my RPIs to make it work. Maybe we should allow for the options to be passed through an environment variable?

@WillooWisp
Copy link
Author

Not sure, but being able to pass it in as an optional parameter is probably the way to go then.

@WillooWisp
Copy link
Author

Will this be merged in?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants